docs: measure Brotli and the Lucide icons, and explain why we change nothing #206
+29
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 3 commits
November 11, 2026 18:08
…othing The peer feedback said that Lucide icons cannot be tree-shaken and that Brotli could help. Both were measured before changing anything: - 13 icons are imported and 13 icon modules are in the bundle (12.1 kB before minification), so the icon imports stay as they are. - Brotli at the default quality is 6% smaller than gzip, and quality 11 is 15% smaller but takes about a second per request. The VM sends gzip and no Brotli. SUSTAINABILITY.md has the numbers and the decision, VM_SETUP.md has the optional steps for Apache (not tried on the VM), and AI_USAGE.md has the entry. Refs #203
…untu and Apache A container test with Ubuntu 26.04.1 and Apache 2.4.66 (the VM's versions) showed that AddOutputFilterByType BROTLI_COMPRESS, the example in the mod_brotli documentation, is not enough on Ubuntu: deflate.conf adds the gzip filter for the same types, and a browser then gets gzip. Brotli is only sent to a client that asks for nothing but br. SetOutputFilter BROTLI_COMPRESS;DEFLATE in a LocationMatch for /project2/assets/ gives Brotli to a browser, gzip to a gzip-only client, and leaves the WOFF2 fonts alone. The module comes with the apache2 package on Ubuntu, so nothing has to be installed. Refs #203
Brotli is not set up, so the steps do not belong in the VM setup guide. The decision and the finding about SetOutputFilter stay in SUSTAINABILITY.md. Refs #203
4 tasks
Sign in
to join this conversation on GitHub.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #203.
The peer feedback said Lucide icons cannot be tree-shaken and that Brotli could help. We measured both and changed no code. The VM is not changed.
curl).SetOutputFilter BROTLI_COMPRESS;DEFLATE.AddOutputFilterByType BROTLI_COMPRESSalone gave gzip to browsers in a container with the VM's Ubuntu and Apache versions. Not tried on the VM itself.Changed:
docs/SUSTAINABILITY.md(numbers and the decision) anddocs/AI_USAGE.md.🤖 Generated with Claude Code